Feat: 파일 관련 기능 유효성 검사 추가 및 S3 버킷 정책 수정 (#221) #242
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 이제 S3 오브젝트를 퍼블릭하게 읽을 수 있도록 허용
- EntityType, EntityId에 @notblank 적용
- EntityType, EntityId로 해당 데이터의 존재 유무를 체크
- 중복되는 로직 메서드화 -> 중복제거 - entityType, entityId로 데이터가 존재하는지 체크하는 로직 추가
- 파일 업로드 시, 파일 입력을 필수로 지정
- Catfe 프론트 운영 서버 추가
- 파일 업로드 성공한 경우 - 파일 업로드 실패한 경우 - 업로드 시, 파일이 없는 경우
- multiparFile 필드 만 @NotNull로 지정
github-actions
bot
changed the title
- 기존 : withport(8001) -> 8001포트에 고정 - 변경 : 할당 withport(0) -> 사용 가능한 랜덤포트 자동할당
문제 - 컨트롤러 테스트에서 S3Mock에 사용하는 포트를 점유하는 문제 발생 -> 서비스 테스트에서 S3Mock을 생성하지 못함 해결 - 컨트롤러 테스트가 끝나면 S3Mock의 포트를 해체 시켜 해당 문제 해결
jueunk617
approved these changes
Oct 12, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 개요
🔨 작업 내용
S3 버킷 정책 수정
GetObject권한 부여PutObject,DeleteObject등 쓰기/삭제 권한은 부여하지 않음resources/policy경로에 추가파일 업로드 유효성 검사 추가
multipartFile필수 입력으로 변경@Valid및@NotNull어노테이션 추가파일 업로드 컨트롤러 테스트 코드 생성
🔗 관련 이슈
Closes #221
📝 참고 사항
✅ 체크리스트